-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Competence Document KPI table #201
Merged
kallepronk
merged 16 commits into
feat/competence-document-profile
from
feat/competence-document-kpi-table
Jan 11, 2024
Merged
Add Competence Document KPI table #201
kallepronk
merged 16 commits into
feat/competence-document-profile
from
feat/competence-document-kpi-table
Jan 11, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…all deliverd assignments are put behind each criteria, got to create a filter/check for this
…ProfileComponent)
…s from the CompetenceProfileComponent class
All submissions per outcome are currently added to the competence document when downloading. TODO: 1. Add grades to each submission 2. Clean up all unneeded code
… document generation
…rdDocument function type
NealGeilen
changed the title
Feat/competence document kpi table
Add Competence document KPI table
Jan 8, 2024
NealGeilen
changed the title
Add Competence document KPI table
Add Competence Document KPI table
Jan 8, 2024
NealGeilen
requested changes
Jan 9, 2024
'?' behind each 'body' element. Added RunProperties (underline) to each submission hyperlink. Want to add a color to each hyperlink, but this makes the grading icons the same color as the text. So it makes you unable to know what each icon means.
NealGeilen
approved these changes
Jan 9, 2024
kallepronk
changed the base branch from
develop
to
feat/competence-document-profile
January 10, 2024 11:00
…document-kpi-table # Conflicts: # Epsilon/Components/CompetenceProfileComponent.cs # Epsilon/Services/CompetenceDocumentService.cs
kallepronk
merged commit Jan 11, 2024
94a0ece
into
feat/competence-document-profile
11 checks passed
NealGeilen
added a commit
that referenced
this pull request
Jan 25, 2024
* add base profile component * add competence profile single axis table generation * change font size and singleaxistable row height * add base profile component * add competence profile single axis table generation * change font size and singleaxistable row height * apply suggestions from code review Neal add changes to get through validation Co-authored-by: Neal Geilen <[email protected]> * Small bugfix * temporary fix for competence profile * Add Competence Document KPI table (#201) * Update * Created the generation of the criteria into the KpiTable + currently all deliverd assignments are put behind each criteria, got to create a filter/check for this * Made allOutcomes distinct * added submission filtering based on available criteria (in CompetenceProfileComponent) * Moved submission filtering up into the CompetenceDocumentService class from the CompetenceProfileComponent class * Added working submission gathering foreach outcome All submissions per outcome are currently added to the competence document when downloading. TODO: 1. Add grades to each submission 2. Clean up all unneeded code * Deleted commented code + added grade per submission to the competence document generation * Run Lint and format frontend + remove '?' from KpiTable class AddToWordDocument function type * Renamed 'KpiTable' to 'KpiTableComponent' + fixed CI errors * Resolved requested changes '?' behind each 'body' element. Added RunProperties (underline) to each submission hyperlink. Want to add a color to each hyperlink, but this makes the grading icons the same color as the text. So it makes you unable to know what each icon means. --------- Co-authored-by: Neal Geilen <[email protected]> Co-authored-by: Kalle Pronk <[email protected]> * add title to kpi table * center competence profile * fix unit test bug * Add Competence Document KPI matrix (#202) * Add KpiMatrixAssignment.cs * Add KpiMatrixOutcome.cs * Add KpiMatrixOutcomeStatus.cs * Add KpiMatrixCollection.cs * Remove old repo code (incorrectly added) * Add KpiMatrixComponent.cs * Rewrite data format (not done) * Add component to CompetenceDocumentService.cs * Remove old KpiMatrixCollection.cs * Fix naming issues in kpi matrix component * Fix data format issues in getLegend method * Add outcomes to components * Implement outcomes in KpiMatrixComponent.cs * Change get color logic * Fix color issues in kpi matrix cells * Fix spacing and remove comments * Fix unit tests with new parameter * Fix unit test issue in kpi matrix component * Implement PR feedback * Implement PR feedback * Add competence profile component create table cell method * Implement create table cell method * Fix borders in kpi matrix * Add "Val" parameter for shading * Add "Val" property to shading * Apply shading directly to TableCellProperties * Add default color values for shading * Fix shading and border errors * Revert "Fix shading and border errors" This reverts commit 27210e6. * Change order of borders * Add distinct filtering in criteria * Fix error for unit tests * Add order by functionality * fix kpi matrix header row bug * fix merge request bugs --------- Co-authored-by: Neal Geilen <[email protected]> Co-authored-by: Bas Bakens <[email protected]> Co-authored-by: Kalle Pronk <[email protected]> * inital version new structure * continue abstraction of word document generation * add correct data to competence profile * fix merge errors * Rewritten unittests * Error fixes * Optimized tests * Load improvements Co-authored-by: Jelle Maas <[email protected]> * Forced nulls on DomainOutcomes from database * Set default for title placement * Set default for title placement * Added Legend for KpiMatrixComponent.cs * Removed unused using * Style change --------- Co-authored-by: Neal Geilen <[email protected]> Co-authored-by: Bas Bakens <[email protected]> Co-authored-by: SaamieVincken <[email protected]> Co-authored-by: Bas Bakens <[email protected]> Co-authored-by: Jelle Maas <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.